Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added an Implementation of MurmurHash2 #18

Merged
merged 2 commits into from
Sep 22, 2018
Merged

Added an Implementation of MurmurHash2 #18

merged 2 commits into from
Sep 22, 2018

Conversation

dutor
Copy link
Contributor

@dutor dutor commented Sep 21, 2018

This is an implementation of MurmurHash2, which is identical to std::hash(at least until GCC 8.1).
This one is more performant on short strings, because:

  1. It could be inlined.
  2. It utilizes the loop unrolling trick.
  3. Besides, it works with the plain old raw bytes array!

This is an implementation of MurmurHash2,
which is identical to `std::hash`(at least until GCC 8.1).
This one is more performant on short strings, because:
  1. It could be inlined.
  2. It utilizes the loop unrolling trick.
  3. Besides, it works with the plain old raw bytes array!
@nebula-community-bot
Copy link
Member

Build succeeded.

Copy link
Member

@sherman-the-tank sherman-the-tank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job!!!

@nebula-community-bot
Copy link
Member

Build succeeded.

@dutor dutor merged commit d96fc92 into vesoft-inc:master Sep 22, 2018
@dutor dutor deleted the murmurhash2 branch September 22, 2018 03:31
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
This is an implementation of MurmurHash2,
which is identical to `std::hash`(at least until GCC 8.1).
This one is more performant on short strings, because:
  1. It could be inlined.
  2. It utilizes the loop unrolling trick.
  3. Besides, it works with the plain old raw bytes array!
yixinglu added a commit to yixinglu/nebula that referenced this pull request Mar 21, 2022
* Cleanup package scripts

* improve clang-format

* Format

* Format

* Check clang home exist

Co-authored-by: yixinglu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants